Skip to content

improve: fix pyproject.toml optional-deps key conflict and clean .gitignore#39

Merged
Coding-Dev-Tools merged 4 commits into
mainfrom
improve/configdrift-20250630
Jul 3, 2026
Merged

improve: fix pyproject.toml optional-deps key conflict and clean .gitignore#39
Coding-Dev-Tools merged 4 commits into
mainfrom
improve/configdrift-20250630

Conversation

@Coding-Dev-Tools

Copy link
Copy Markdown
Owner

Automated improvement by dev-engineer.

  • Renamed [project.optional-dependencies] license to license-check to avoid PEP 621 metadata key conflict with [project] license
  • Removed stray nul entry from .gitignore (Windows artifact)

@github-actions

github-actions Bot commented Jun 30, 2026

Copy link
Copy Markdown

🤖 Automated Code Review

✅ Ruff Lint — No issues

⚠️ Ruff Format — Formatting needed

Would reformat: src/configdrift/cli.py
Would reformat: src/configdrift/diff.py
Would reformat: src/configdrift/loader.py
Would reformat: tests/test_cli.py
Would reformat: tests/test_coverage_gaps.py
Would reformat: tests/test_diff.py
Would reformat: tests/test_loader.py
7 files would be reformatted, 2 files already formatted

✅ Secret Detection — Clean

✅ Large Files — Within limits

📊 Diff Stats — 3 file(s) changed

 .github/workflows/ci.yml | 6 ++----
 .gitignore               | 9 ---------
 pyproject.toml           | 2 +-
 3 files changed, 3 insertions(+), 14 deletions(-)

Verdict: ⚠️ Warnings — Lint/format issues found. Recommend fixing before merge.

Automated by Coding-Dev-Tools/.github reusable workflow.

Coding-Dev-Tools and others added 3 commits July 2, 2026 00:16
Add a __main__ guard so 'python -m configdrift.cli' (what cli.js calls)
actually runs the app. Pin checkout/setup-python to SHAs like the rest
of the suite.
@Coding-Dev-Tools Coding-Dev-Tools merged commit 8c272b5 into main Jul 3, 2026
@Coding-Dev-Tools Coding-Dev-Tools deleted the improve/configdrift-20250630 branch July 3, 2026 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant